-
Notifications
You must be signed in to change notification settings - Fork 173
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Correction for the method SubCircuit.check_nodes() #136
Open
jmgc
wants to merge
141
commits into
PySpice-org:master
Choose a base branch
from
jmgc:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Change of the definition of Node to ensure the node name is always stored in lower case. Converted the _external_nodes in SubCircuit into a tuple of Nodes. Modified SubCircuit.check_nodes to use the Node name property, and a dictionary to verify if a node has more than one element connected.
Remove unused line.
Schemdraw
Thanks to help to improve PySpice ! Main concern is to find a way to decouple the netlist and the schematic stuff so as to keep PySpice independent of the simulator and the schematic renderer. Must found some time to work on this and the latest issues ! Will try during Christmas week ! |
Adding the possibility to use parameters with the spice name in an element.
Add the management of expressions inside brackets.
Avoding .idea.
Detected an issue with the parameters determination. The grammar has been modified to be more robust and tests have been added.
Changed the unit suffixes to lower case, and commented those that can be misinterpreted when the units in the spice file are converted to lower case. Meter left, though it can be misinterpreted with mili.
To avoid issues with Spice, the full spice code is lower cased. Also revised the expression parsing to avoid issues between the expressions, the units and the recovering of the Spice code.
Work with lower case in all the cases, to be consistent with SPICE's case-insensitive behaviour.
Rename celsius SPICE suffix.
Manage an expression as positional parameter.
Correction of unit name.
Update PrefixedUnit and UnitValue to manage expressions.
Take into account the case when the Unit value is already an Expression. And the associated test.
Correction to avoid circular imports.
Update the example.
It is subsided by EBNFParser.
Updates on Library and Netlist to allow the use of EBNFParser in library.
Solved issues with the inclusion of the spice.
Change to numpy frombuffer to avoid warning.
Improve the errors information.
Improve the errors detection.
Updated to use python 311
Added a new test to confirm that multiple elements are read.
Added a check to avoid loosing files data.
Correction of the method an associated test.
Improve docs.
Added tatsu requirement and update python version.
Solve issue with raw streams.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I have made some slight changes to correct the check_nodes method in SubCircuit, and convert a SubCircuit node in an object of the class Node so the code can be reused.
Thanks in advance